Skip to content

Declare pytest-timeout as a test dependency#652

Closed
nathanjmcdougall wants to merge 1 commit intopypest:developfrom
nathanjmcdougall:fix/declare-pytest-timeout
Closed

Declare pytest-timeout as a test dependency#652
nathanjmcdougall wants to merge 1 commit intopypest:developfrom
nathanjmcdougall:fix/declare-pytest-timeout

Conversation

@nathanjmcdougall
Copy link
Copy Markdown
Contributor

@nathanjmcdougall nathanjmcdougall commented Dec 12, 2025

It's being used here, for example:

pyemu/autotest/utils_tests.py

Lines 826 to 827 in 5dd3c78

@pytest.mark.timeout(method='thread', timeout=90)
def geostat_prior_builder_test(tmp_path):

At the moment, this isn't declared as a dependency so when trying to collect the pytest tests, a warning is emitted:

>> pytest autotest --collect-only 

...

autotest\utils_tests.py:826
  C:\Users\namc\Repositories\pyemu\autotest\utils_tests.py:826: PytestUnknownMarkWarning: Unknown pytest.mark.timeout - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html

@mwtoews
Copy link
Copy Markdown
Contributor

mwtoews commented Dec 13, 2025

Same fix in #643

@briochh
Copy link
Copy Markdown
Collaborator

briochh commented Jan 6, 2026

now covered off by #668

@briochh briochh closed this Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nathanjmcdougall @mwtoews @briochh